Auto merge of #3154 - steveklabnik:gh3124, r=alexcrichton
authorbors <bors@rust-lang.org>
Fri, 7 Oct 2016 20:09:27 +0000 (13:09 -0700)
committerGitHub <noreply@github.com>
Fri, 7 Oct 2016 20:09:27 +0000 (13:09 -0700)
commite7523cd5ad46bcd8800f418f996e81fd2d2800b0
treea9e111108df1445eae5282b1b5ca43220c9c84f5
parent09a955cfcb7db06bfe805894b7d29e5c5922552d
parent464e6a96fb599049761a8842c89118bde1683edd
Auto merge of #3154 - steveklabnik:gh3124, r=alexcrichton

upgrade semver

This is a spike at fixing #3124.

@alexcrichton I'm not sure how to actually emit the error message here. In the TOML example you linked me: https://github.com/rust-lang/cargo/blob/5593045ddef2744c1042dee0c0037c2ebcc1834e/src/cargo/util/toml.rs#L166

That takes a Config as an argument. This function does not. What's the right approach here?

Also, this code is messy. I am 99% sure I can write it nicer with combinators. This is just to get the discussion going.